Skip to content

Refactor: modernize Sealed module#192

Closed
erayaydin wants to merge 1 commit intomainfrom
refactor/sealed-module
Closed

Refactor: modernize Sealed module#192
erayaydin wants to merge 1 commit intomainfrom
refactor/sealed-module

Conversation

@erayaydin
Copy link
Copy Markdown
Member

Add types for Sealed module components. Replace static usage with the const. Use str_starts_with instead of substr function. Add explicit imports for the exception classes. Add PHPDoc to all sealed components. Add new test for coverage of getDecryptionKey method.

Add types for Sealed module components. Replace `static` usage with the
`const`. Use `str_starts_with` instead of `substr` function. Add
explicit imports for the exception classes. Add PHPDoc to all sealed
components. Add new test for coverage `getDecryptionKey` method.

Related-Task: INTER-1490
@github-actions
Copy link
Copy Markdown
Contributor

⚠️ This PR doesn't contain any changesets. If there are user-facing changes, don't forget to run:

pnpm exec changeset

to create a changeset.

@github-actions
Copy link
Copy Markdown
Contributor

⚠️ Template files and generated files are not in-sync.

Following files do not match corresponding templates:
src/Sealed/DecompressionException.php src/Sealed/InvalidSealedDataException.php src/Sealed/InvalidSealedDataHeaderException.php src/Sealed/Sealed.php src/Sealed/UnsealAggregateException.php src/Sealed/UnsealException.php

To fix this warning, make sure template files are up-to-date, and generate files by running the following command:

bash ./scripts/generate.sh

@github-actions
Copy link
Copy Markdown
Contributor

🟢 Coverage increased by 35.93%

Code Coverage Report

Coverage Report

St. Category Percentage Covered / Total
🔴 Statements 35.93% 1836 / 5110
🔴 Functions 28.23% 670 / 2373
Files Coverage
St. File Methods Statements Total Coverage
🔴 src/Model/BotdBotResult.php 0.00% 0.00% 0.00%
🔴 src/Model/ErrorCode.php 0.00% 0.00% 0.00%
🔴 src/Model/ModelInterface.php 0.00% 0.00% 0.00%
🔴 src/Model/ProxyConfidence.php 0.00% 0.00% 0.00%
🔴 src/Model/VPNConfidence.php 0.00% 0.00% 0.00%

@erayaydin
Copy link
Copy Markdown
Member Author

Closing this PR. global_namespace_import php-cs-fixer rule change touches/changes all source files and is better suited for the v4 migration.

@erayaydin erayaydin closed this Mar 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant